Skip to content

Feature/bi 2390#167

Open
cvacalares wants to merge 6 commits intodevelopfrom
feature/BI-2390
Open

Feature/bi 2390#167
cvacalares wants to merge 6 commits intodevelopfrom
feature/BI-2390

Conversation

@cvacalares
Copy link
Collaborator

Description

BI-2390 - Program Administrator can see a delete button next to each collaborator
Add step definitions for checking Collaborator's delete button.

Testing

Please include a link to a successful run of TAF for this change

Checklist:

  • [x ] I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation

Add function isDeleteVisible for checking the delete button of the collaborator.
@cvacalares cvacalares requested review from a team, dmeidlin and mlm483 and removed request for a team June 29, 2025 18:55
Comment on lines 54 to 57
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
And user can see "Experimental Collaborator1(experimentcollaborator1@mailinator.com)" as Collaborator
And user can see "Experimental Collaborator2(experimentcollaborator2@mailinator.com)" as Collaborator
And user can see "Experimental Collaborator1(experimentcollaborator1@mailinator.com)" delete button of Collaborator
And user can see "Experimental Collaborator2(experimentcollaborator2@mailinator.com)" delete button of Collaborator
Then user can see "Experimental Collaborator1(experimentcollaborator1@mailinator.com)" as Collaborator
And user can see "Experimental Collaborator2(experimentcollaborator2@mailinator.com)" as Collaborator
And user can see "Experimental Collaborator1(experimentcollaborator1@mailinator.com)" delete button of Collaborator
And user can see "Experimental Collaborator2(experimentcollaborator2@mailinator.com)" delete button of Collaborator

I think this is more idiomatic gherkin.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@cvacalares
Copy link
Collaborator Author

@mlm483 - Done.

@cvacalares cvacalares requested a review from mlm483 September 3, 2025 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments